Skip to content

Update Tool Version for mavenExtract and nugetExtract#631

Merged
elrayle merged 2 commits intoclearlydefined:masterfrom
yashkohli88:yk/increment-tool-version-cd
Jan 29, 2025
Merged

Update Tool Version for mavenExtract and nugetExtract#631
elrayle merged 2 commits intoclearlydefined:masterfrom
yashkohli88:yk/increment-tool-version-cd

Conversation

@yashkohli88
Copy link
Copy Markdown
Contributor

Tasks Completed:

  • Class Identification: Identified the extract classes that depend on the extractLicenseFromLicenseUrl method, which utilizes the _licenseUrlOverrides list.
  • Tool Version Updates: Updated tool versions within the relevant extract classes for both Maven and NuGet.
  • Test Case Addition: Added a test case to validate and verify the updated versions, ensuring the re-harvesting is possible for coordinates of nuget and maven packages.
  • Manual Re-harvesting: Manually re-harvested Maven and NuGet components on a local system. This process confirmed that the ClearlyDefined tool processes these components as expected after updating the tool version, while other tools for these components are appropriately excluded.

This PR is related to #630
Find the relevant comment and discussion here clearlydefined/service#1246 (comment)

Copy link
Copy Markdown
Collaborator

@elrayle elrayle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the link the related PR to provide context.

@elrayle elrayle merged commit d5fa7c6 into clearlydefined:master Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants